Skip to content

test(ci): register microsoft-designer-web-6672 in stryker tap.testFiles - #7657

Closed
diegosouzapw wants to merge 1 commit into
release/v3.8.49from
test/fix-mutation-coverage-drift-6672
Closed

diegosouzapw wants to merge 1 commit into
release/v3.8.49from
test/fix-mutation-coverage-drift-6672

Conversation

@diegosouzapw

Copy link
Copy Markdown
Collaborator

Problem

check:mutation-test-coverage --strict (the Fast Quality Gates step) was base-red on every open release PR — the last unexplained base-red.

Root cause

The drift guard's invariant: every tests/unit/** test that imports a mutated module (stryker.conf.json mutate) MUST be listed in tap.testFiles. Today's #7609 (Microsoft Designer provider) added tests/unit/microsoft-designer-web-6672.test.ts, which imports resolvePublicCred from the mutated module open-sse/utils/publicCreds.ts — but the test was never added to tap.testFiles. The guard fails --strict on exactly that drift.

Fix

Add the test to stryker.conf.json tap.testFiles (alphabetical slot). This is the fix the gate's own docs prescribe ("Add them to tap.testFiles").

+ "tests/unit/microsoft-designer-web-6672.test.ts",

Validation

node scripts/check/check-mutation-test-coverage.mjs --strict
✓ No drift — every covering unit test is listed in tap.testFiles.   (exit 0)

Config-only — no test or production code touched. The file exists only on release/v3.8.49 (added by #7609), so no main companion is needed. Clears the last shared base-red on Fast Quality Gates for all release PRs.

…stFiles

check:mutation-test-coverage --strict (Fast Quality Gates) was base-red on
every release PR: the unit test tests/unit/microsoft-designer-web-6672.test.ts
(added by #7609, Microsoft Designer provider) imports the mutated module
open-sse/utils/publicCreds.ts but was never added to stryker.conf.json
tap.testFiles. The drift guard fails --strict on exactly this.

Add the test to tap.testFiles (alphabetical slot). Advisory + --strict both
report zero drift after. Config-only; no test or production code touched.
@gemini-code-assist

Copy link
Copy Markdown
Contributor

Caution

The consumer version of Gemini Code Assist on GitHub has been sunset. All code review activity has officially ceased.

@diegosouzapw

Copy link
Copy Markdown
Collaborator Author

Duplicata (subconjunto) do #7652, que traz a mesma linha do stryker mais o fragmento de changelog exigido pelo docs-sync gate. Fechando em favor do #7652 assim que ele mergear.

@diegosouzapw

Copy link
Copy Markdown
Collaborator Author

Fechando como duplicata de #7652, que registra exatamente o mesmo microsoft-designer-web-6672.test.ts no stryker.conf.json tap.testFiles e ainda inclui o changelog fragment. O #7652 destrava o mesmo base-red do Fast Quality Gates. Obrigado — o fix vai pela outra PR.

@diegosouzapw
diegosouzapw deleted the test/fix-mutation-coverage-drift-6672 branch July 19, 2026 21:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant